-
Notifications
You must be signed in to change notification settings - Fork 0
PR de Correção #23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: correcao-projeto
Are you sure you want to change the base?
PR de Correção #23
Conversation
Postman, routers and screens
Component CardPokemon
Style page details
Router, Pagina de Erro, e Estilização Error
Home/card Estilizados, Espaçamento no Headear
Style atual details
Utilização do global state, renderização dos pokemons
Poke allison
Botões de acordo com a Pagina
Estilização do Button
Function do goToPage por props antigo
Renderização dos pokemons escolhidos na pokedex
Adicionada a função de remover da pokedex
Botões de Add e Remove
Animação de loading
Funcionalidades
Primeira revisão no código
Ajuste da fonte para pegar em todo o site
… Button do Menu | Organizando Componente do Button | e entre outro :)
Poke allison
Details button
leticia-chijo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Olá! O projeto de vocês foi considerado acima do esperado, parabéns! 🎉
O layout está bonito e o site funciona do jeito que foi pedido, além de ser responsivo. Algumas observações:
- Na página da pokédex, quando eu ainda não adicionei nenhum pokémon, ele fica carregando infinitamente. Acho que nessa tela, teoricamente, nem precisa colocar loading! Esses dados são locais, não tem nenhuma requisição para pegar eles, então ele não deveria demorar para aparecer. Um placeholder de "Você ainda não tem nenhum pokémon, adicione!" ou algo assim seria mais apropriado nesse caso, só pra não deixar vazio mesmo.
- Vocês deixaram um PR aberto! O conteúdo dele não foi considerado para a correção, apenas o que está na master. Lembrem de fechar tudo nos próximos 😄
- O código está bem organizado e gostei de como vocês guardaram as informações
Acho que é isso! Se algum ponto ainda não ficou claro, assistam ao vídeo com a sugestão de resolução que enviamos na sexta e, se sobrar alguma dúvida, por favor mandem lá no canal! ❤️
PR de Correção